Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kim file permissions #627

Merged
merged 2 commits into from
Aug 3, 2022

Conversation

gowthamsk-arm
Copy link
Contributor

Closes #598.

The current key info manager files are not having any permissions set on them. This MR adds suitable permissions to directories and files of the KIM and also adds unit tests to verify if the permissions set are correct.

Signed-off-by: Gowtham Suresh Kumar gowtham.sureshkumar@arm.com

@gowthamsk-arm
Copy link
Contributor Author

The discussion about the KIM file permissions for parsec user and group is still ongoing. This MR can be merged once we agree on a particular approach.
#598

@gowthamsk-arm gowthamsk-arm force-pushed the kim_file_permissions branch 3 times, most recently from 6661fd1 to 3e13281 Compare August 2, 2022 10:32
Copy link
Member

@ionut-arm ionut-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 👍🏻

Signed-off-by: Gowtham Suresh Kumar <gowtham.sureshkumar@arm.com>
Signed-off-by: Gowtham Suresh Kumar <gowtham.sureshkumar@arm.com>
@ionut-arm ionut-arm merged commit 05f4e9f into parallaxsecond:main Aug 3, 2022
@gowthamsk-arm gowthamsk-arm deleted the kim_file_permissions branch September 5, 2022 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong permissions on KIM files
3 participants